Skip to content

Conversation

AlexandreSinger
Copy link
Contributor

Found that the CI was using a separate script which was indirectly calling the standard package install script. This can cause the standard package install script to regress, but it was also causing pressure on the GitHub runners by having them install more packages than they needed.

@github-actions github-actions bot added infra Project Infrastructure scripts Utility & Infrastructure scripts lang-shell Shell scripts (bash etc.) labels Aug 23, 2025
Found that the CI was using a separate script which was indirectly
calling the standard package install script. This can cause the standard
package install script to regress, but it was also causing pressure on
the GitHub runners by having them install more packages than they
needed.
@AlexandreSinger AlexandreSinger changed the title [WIP][CI] Moved to Use Standard Package Installer [CI] Moved to Use Standard Package Installer Script Aug 24, 2025
Copy link
Contributor

@AmirhosseinPoolad AmirhosseinPoolad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR Alex. The changes sound very good to me. Had a couple of thoughts about some not super important things. Looks good otherwise.

@AlexandreSinger AlexandreSinger merged commit 0511c6d into master Aug 26, 2025
30 checks passed
@AlexandreSinger AlexandreSinger deleted the feature-ci-packages branch August 26, 2025 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra Project Infrastructure lang-shell Shell scripts (bash etc.) scripts Utility & Infrastructure scripts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants